use Work::noop() more

Using Work::noop() in these places is clearer.

diff --git a/src/cargo/ops/cargo_rustc/custom_build.rs b/src/cargo/ops/cargo_rustc/custom_build.rs
index c53fc612e62ce1f240945a0b1008fd3c836ad55a..727f0d324bff32c6b6b9893b9d3610acb32e173e 100644 (file)
--- a/src/cargo/ops/cargo_rustc/custom_build.rs
+++ b/src/cargo/ops/cargo_rustc/custom_build.rs
@@ -79,7 +79,7 @@ pub fn prepare<'a, 'cfg>(cx: &mut Context<'a, 'cfg>, unit: &Unit<'a>)
                                     unit.pkg, unit.target.name()));
     let overridden = cx.build_state.has_override(unit);
     let (work_dirty, work_fresh) = if overridden {
-        (Work::new(|_| Ok(())), Work::new(|_| Ok(())))
+        (Work::noop(), Work::noop())
     } else {
         build_work(cx, unit)?
     };

diff --git a/src/cargo/ops/cargo_rustc/mod.rs b/src/cargo/ops/cargo_rustc/mod.rs
index 34601913ac7a799fb48dc16c502fc12baa133809..873bed3e785aedce7a43187a6a2c8c00d70b0872 100644 (file)
--- a/src/cargo/ops/cargo_rustc/mod.rs
+++ b/src/cargo/ops/cargo_rustc/mod.rs
@@ -234,7 +234,7 @@ fn compile<'a, 'cfg: 'a>(cx: &mut Context<'a, 'cfg>,
         custom_build::prepare(cx, unit)?
     } else if unit.profile.doc && unit.profile.test {
         // we run these targets later, so this is just a noop for now
-        (Work::new(|_| Ok(())), Work::new(|_| Ok(())), Freshness::Fresh)
+        (Work::noop(), Work::noop(), Freshness::Fresh)
     } else {
         let (mut freshness, dirty, fresh) = fingerprint::prepare_target(cx, unit)?;
         let work = if unit.profile.doc {